Skip to content

Add Codable conformance and human-readable names to EncryptionCipher#21

Merged
nedithgar merged 2 commits intomainfrom
feat/add-Codable-to-EncryptionCipher
Oct 14, 2025
Merged

Add Codable conformance and human-readable names to EncryptionCipher#21
nedithgar merged 2 commits intomainfrom
feat/add-Codable-to-EncryptionCipher

Conversation

@nedithgar
Copy link
Copy Markdown
Owner

Enhance the EncryptionCipher struct with Codable conformance for easier serialization and add human-readable names for better display in UI contexts. Update the version to 0.1.9.

Copilot AI review requested due to automatic review settings October 14, 2025 04:00
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the EncryptionCipher struct by adding Codable conformance for JSON serialization and introducing human-readable display names for UI contexts. The version is also updated from 0.1.8 to 0.1.9 to reflect these changes.

Key changes:

  • Added Codable conformance to EncryptionCipher for serialization support
  • Implemented humanReadableName property with user-friendly cipher names
  • Updated version number to 0.1.9

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Sources/SwiftKeyGenCLI/main.swift Version number updated to 0.1.9
Sources/SwiftKeyGen/Formats/OpenSSH/OpenSSHPrivateKey.swift Enhanced EncryptionCipher with Codable conformance and human-readable names

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nedithgar nedithgar merged commit 1e32f4f into main Oct 14, 2025
4 checks passed
@nedithgar nedithgar deleted the feat/add-Codable-to-EncryptionCipher branch October 14, 2025 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants